Merged
Conversation
…s which have not yet been made
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Pull Request Overview
This PR merges multiple changes across the codebase to update UI components and API endpoints in preparation for the final product release. Key changes include an updated footer with new links and styling, enhancements to editor components with refined content management in the ContentArea, and multiple updates to API routes and user forms for improved error handling and user feedback.
Reviewed Changes
Copilot reviewed 33 out of 36 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/components/Footer.tsx | Updated footer links to use Link components with revised styling. |
| src/components/Editor/EditTools.tsx | Added and refactored editor toolbar actions. |
| src/components/Editor/ContentArea.tsx | Introduced state update for editor HTML content. |
| src/components/DisclaimerWarning.tsx | Enhanced disclaimer with an additional link. |
| src/app/page.tsx | Modified homepage sections and updated hero content. |
| src/app/find-out-more/page.tsx | Added an informational page about the project. |
| src/app/app/page.tsx | Created a new app page with left menu and middle component. |
| src/app/api/auth/[...nextauth]/options.ts | Adjusted auth options to throw errors instead of returning null. |
| src/app/api/(user)/signup/route.ts | Updated the signup endpoint with refined responses and comments. |
| src/app/api/(Post)/getPost/route.ts | Implemented pagination and enhanced error handling for posts. |
| src/app/api/(Post)/createpost/route.ts | Added a new endpoint for creating posts. |
| src/app/(User)/signup/SignUpForm.tsx | Updated signup form with additional comments and form hints. |
| src/app/(User)/signin/SignInForm.tsx | Enhanced signin form with session checks and improved messaging. |
| src/app/(User)/forgotpassword/ForgotPasswordForm.tsx | Updated forgot password form to redirect upon success. |
| src/ZodSchema/UserSchema.ts | Revised Zod schemas with added comments. |
| next.config.mjs | Removed experimental serverActions configuration. |
Files not reviewed (3)
- components.json: Language not supported
- package.json: Language not supported
- prisma/schema.prisma: Language not supported
Comment on lines
+18
to
+19
| <li><Link href="/about" className='text-decoration: hover:underline hover:text-blue-500'>Our Mission</Link></li> | ||
| <li><Link href="/contact" className='text-decoration: hover:underline hover:text-blue-500'>Contact Us</Link></li> |
There was a problem hiding this comment.
The className contains 'text-decoration:' which is not a valid Tailwind utility. Consider removing 'text-decoration:' to correctly apply hover styles.
Suggested change
| <li><Link href="/about" className='text-decoration: hover:underline hover:text-blue-500'>Our Mission</Link></li> | |
| <li><Link href="/contact" className='text-decoration: hover:underline hover:text-blue-500'>Contact Us</Link></li> | |
| <li><Link href="/about" className='hover:underline hover:text-blue-500'>Our Mission</Link></li> | |
| <li><Link href="/contact" className='hover:underline hover:text-blue-500'>Contact Us</Link></li> |
Comment on lines
+64
to
+66
| const html = editor.getHTML() | ||
| setContent(html) | ||
|
|
There was a problem hiding this comment.
Calling setContent during render may trigger an infinite re-render loop; consider using a useEffect hook to update state based on the editor's HTML output.
Suggested change
| const html = editor.getHTML() | |
| setContent(html) | |
| React.useEffect(() => { | |
| if (editor) { | |
| const updateContent = () => { | |
| const html = editor.getHTML() | |
| setContent(html) | |
| } | |
| editor.on('update', updateContent) | |
| return () => { | |
| editor.off('update', updateContent) | |
| } | |
| } | |
| }, [editor]) |
Comment on lines
+121
to
+122
| <p className='text-centre'>You are currently logged in to log in with another account.</p> | ||
| <p className='text-centre'>To continue please first logout.</p> |
There was a problem hiding this comment.
The Tailwind CSS class 'text-centre' is misspelled; it should be 'text-center'.
Suggested change
| <p className='text-centre'>You are currently logged in to log in with another account.</p> | |
| <p className='text-centre'>To continue please first logout.</p> | |
| <p className='text-center'>You are currently logged in to log in with another account.</p> | |
| <p className='text-center'>To continue please first logout.</p> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Doesn't live up to expecations but time is a thing.